Skip to content

STT 입력 개선 - #296

Merged
hyoj-dev merged 2 commits into
developfrom
feat/stt-improve
Aug 17, 2026
Merged

STT 입력 개선#296
hyoj-dev merged 2 commits into
developfrom
feat/stt-improve

Conversation

@hyoj-dev

Copy link
Copy Markdown
Collaborator

Summary

  • V2 메시지 API에 inputType(TEXT/STT)을 추가하고 기본값을 TEXT로 설정
  • STT 메시지 중복 검증 및 입력 타입 전달 지원
  • 음성 변환 실패 로깅과 공통 예외 처리를 강화
  • 종료·삭제된 V2 회고의 STT 요청을 차단
  • 기존 음성 답변 API를 deprecated 처리하고 관련 문서 업데이트
  • 잘못된 요청 본문에 대해 일관된 400 응답 처리

Testing

  • V2 메시지 입력 타입 및 잘못된 입력 타입 검증 테스트 추가
  • STT 변환 실패·재시도·회고 상태 검증 테스트 추가
  • Not run (not requested)

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 24e01182c3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

} catch (exception: SpeechTranscriptionFailedException) {
throw exception
} catch (exception: Exception) {
throw SpeechTranscriptionFailedException("providerFailure: ${exception::class.simpleName}")

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Preserve the provider exception when translating failures

When the STT client throws an unexpected exception such as ResourceAccessException, this replacement exception neither retains the cause nor logs it before wrapping. The outer warning therefore reports only SpeechTranscriptionFailedException, and the global business-exception handler also omits a stack trace, so the original error message and call stack that were previously logged by the generic exception handler are lost during provider outages. Preserve the original exception as the cause or log it with its stack trace before translating it.

Useful? React with 👍 / 👎.

@hyoj-dev
hyoj-dev merged commit a1a4fa6 into develop Aug 17, 2026
3 checks passed
@hyoj-dev
hyoj-dev deleted the feat/stt-improve branch August 17, 2026 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant